Skip to content

Comments

[IcebergIO] Add ITs for RESTCatalog using BLMS#35360

Open
ahmedabu98 wants to merge 5 commits intoapache:masterfrom
ahmedabu98:test_rest_catalog
Open

[IcebergIO] Add ITs for RESTCatalog using BLMS#35360
ahmedabu98 wants to merge 5 commits intoapache:masterfrom
ahmedabu98:test_rest_catalog

Conversation

@ahmedabu98
Copy link
Contributor

@ahmedabu98 ahmedabu98 commented Jun 18, 2025

Fixes #35255

@ahmedabu98 ahmedabu98 marked this pull request as draft June 18, 2025 19:22
@ahmedabu98
Copy link
Contributor Author

ahmedabu98 commented Jun 18, 2025

Catalog currently only assumes JWT tokens.

Still need to find a solution that allows us to pass application default credentials to the catalog

Updated to Iceberg 1.10.0 since, and can use GoogleAuthManager

@github-actions
Copy link
Contributor

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@beam.apache.org list. Thank you for your contributions.

@github-actions github-actions bot added the stale label Aug 18, 2025
@github-actions
Copy link
Contributor

This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

@github-actions github-actions bot closed this Aug 26, 2025
@ahmedabu98 ahmedabu98 reopened this Feb 17, 2026
@github-actions github-actions bot removed the stale label Feb 18, 2026
@ahmedabu98 ahmedabu98 marked this pull request as ready for review February 18, 2026 18:49
@github-actions
Copy link
Contributor

Assigning reviewers:

R: @Abacn for label java.

Note: If you would like to opt out of this review, comment assign to next reviewer.

Available commands:

  • stop reviewer notifications - opt out of the automated review tooling
  • remind me after tests pass - tag the comment author after tests pass
  • waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

@codecov
Copy link

codecov bot commented Feb 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.04%. Comparing base (5d6cb04) to head (5ab72fe).
⚠️ Report is 23 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #35360      +/-   ##
============================================
- Coverage     40.06%   40.04%   -0.02%     
  Complexity     3416     3416              
============================================
  Files          1178     1178              
  Lines        187341   187219     -122     
  Branches       3589     3589              
============================================
- Hits          75053    74975      -78     
+ Misses       108897   108853      -44     
  Partials       3391     3391              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task] [IcebergIO]: Add RESTCatalog tests

1 participant